Skip to content

OIDC/Trusted Publishing (1/2)#513

Merged
interim17 merged 1 commit intomainfrom
feature/oidc
Feb 23, 2026
Merged

OIDC/Trusted Publishing (1/2)#513
interim17 merged 1 commit intomainfrom
feature/oidc

Conversation

@interim17
Copy link
Copy Markdown
Contributor

Our npm tokens are expired, and no one outside of infra was a maintainer anymore, preventing us from publishing a new version on npm.

We're taking this as an opportunity to move to using Trusted Publishing.

Infra says we should be set up on the NPM registry side, and this PR almost exactly mimics the comparable change in vole-app:
https://github.com/allen-cell-animated/vole-app/pull/475/files

I also mirrored the approach of pinning actions to particular commit hashes and bumped both versions on node and actions/setup-node.

I didn't downgrade actions/checkout but did use a hash for the version we were on.

I think to test this we will need to push a new version tag, I think the sequence will be to merge this first and then do a version release.

@interim17 interim17 requested a review from a team as a code owner February 18, 2026 17:58
@interim17 interim17 requested review from ShrimpCryptid and meganrm and removed request for a team February 18, 2026 17:58
@github-actions
Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 22.14% 2583 / 11665
🔵 Statements 22.14% 2583 / 11665
🔵 Functions 25% 149 / 596
🔵 Branches 80.41% 349 / 434
File CoverageNo changed files found.
Generated in workflow #629 for commit 576162e by the Vitest Coverage Report Action

@griffinfujioka griffinfujioka self-requested a review February 19, 2026 17:38
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
- run: npm publish --provenance # Provenance required for Trusted Publishers
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember to update the package.json with the repository URL if it's not already in there

@interim17 interim17 merged commit 8de2c3e into main Feb 23, 2026
6 checks passed
@interim17 interim17 deleted the feature/oidc branch February 23, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants